Skip to content

Conversation

@troglobit
Copy link
Contributor

@troglobit troglobit commented Oct 2, 2025

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe): workflow changes

@troglobit troglobit added the ci:main Build default defconfig, not minimal label Oct 2, 2025
@troglobit troglobit changed the title Misc Build, test, and release workflow fixes Oct 2, 2025
@troglobit troglobit marked this pull request as draft October 2, 2025 00:18
@troglobit troglobit self-assigned this Oct 2, 2025
@troglobit troglobit force-pushed the misc branch 3 times, most recently from 6e557c4 to 86d0aab Compare October 2, 2025 06:10
@troglobit troglobit marked this pull request as ready for review October 2, 2025 11:44
@troglobit troglobit enabled auto-merge October 2, 2025 11:44
@troglobit troglobit requested a review from mattiaswal October 2, 2025 11:45
@troglobit troglobit marked this pull request as draft October 2, 2025 12:16
auto-merge was automatically disabled October 2, 2025 12:16

Pull request was converted to draft

Here we use a check-trigger job that all the others depend on, which
should prevent duplicate workflows starting a bit more elegantly than
killing one of them with the concurrency checker.

Signed-off-by: Joachim Wiberg <[email protected]>
With all jobs now depending on check-trigger we can do the evaluations
there and set some variables that can be reused later.

Signed-off-by: Joachim Wiberg <[email protected]>
Fixes the following issue, see also https://devops.stackexchange.com/a/17853/26590:

make[1]: Entering directory '/home/github-jaffa/actions-runner/_work/infix/infix/buildroot'
/home/github-jaffa/actions-runner/_work/infix/infix/test/env -r -b /home/github-jaffa/actions-runner/_work/infix/infix
/home/github-jaffa/actions-runner/_work/infix/infix/test/9pm/9pm.py -v
/home/github-jaffa/actions-runner/_work/infix/infix/test/case/all-repo.yaml
/home/github-jaffa/actions-runner/_work/infix/infix/test/case/all-unit.yaml
Error: rootlessport conflict with ID 1
make[1]: *** [/home/github-jaffa/actions-runner/_work/infix/infix/test/test.mk:58: test-unit] Error 126
make[1]: Leaving directory '/home/github-jaffa/actions-runner/_work/infix/infix/buildroot'
make: *** [Makefile:31: test-unit] Error 2

Signed-off-by: Joachim Wiberg <[email protected]>
The 'brief' keyword has been gone for a while now.  Also, update the
ouput to match the "new" cli-pretty formatting.

Fixes #1174

Signed-off-by: Joachim Wiberg <[email protected]>
Fixes #1175

Signed-off-by: Joachim Wiberg <[email protected]>
@troglobit troglobit marked this pull request as ready for review October 2, 2025 15:10
@troglobit troglobit requested a review from mattiaswal October 2, 2025 15:55
Copy link
Contributor

@mattiaswal mattiaswal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@troglobit troglobit merged commit e91ffeb into main Oct 2, 2025
6 checks passed
@troglobit troglobit deleted the misc branch October 2, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:main Build default defconfig, not minimal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants